Skip to content

Fix/getcloser security config#346

Merged
soohyunme merged 9 commits intodeploy/cert-developfrom
fix/getcloser-security-config
Feb 11, 2026
Merged

Fix/getcloser security config#346
soohyunme merged 9 commits intodeploy/cert-developfrom
fix/getcloser-security-config

Conversation

@soohyunme
Copy link
Member

๐Ÿ“ PR ํƒ€์ž…

  • ๋ฒ„๊ทธ ์ˆ˜์ • (fix)
  • ์‹ ๊ทœ ๊ธฐ๋Šฅ (feat)
  • ๋ฌธ์„œ ์—…๋ฐ์ดํŠธ (docs)
  • ์ฝ”๋“œ ์Šคํƒ€์ผ / ํฌ๋งทํŒ… (style)
  • ๋ฆฌํŒฉํ† ๋ง (refactor)
  • ํ…Œ์ŠคํŠธ (test)
  • ๊ธฐํƒ€ (chore, ci ๋“ฑ)

๐Ÿ”จ ์ž‘์—… ๋‚ด์šฉ

  • ๋ณด์•ˆ ๊ฐ•ํ™”๋ฅผ ์œ„ํ•ด Backend์˜ CORS ์„ค์ •์„ ๋ณ€๊ฒฝํ–ˆ์Šต๋‹ˆ๋‹ค.

  • ๊ธฐ์กด์˜ ๋ชจ๋“  ์˜ค๋ฆฌ์ง„ ํ—ˆ์šฉ(allow_origins=["*"]) ์„ค์ •์„ ์ œ๊ฑฐํ–ˆ์Šต๋‹ˆ๋‹ค.

  • ํ™˜๊ฒฝ ๋ณ€์ˆ˜ CORS_ORIGINS๋ฅผ ํ†ตํ•ด ํ—ˆ์šฉํ•  ์˜ค๋ฆฌ์ง„์„ ์„ค์ •ํ•  ์ˆ˜ ์žˆ๋„๋ก ๋กœ์ง์„ ์ถ”๊ฐ€ํ–ˆ์Šต๋‹ˆ๋‹ค.

  • ํ™˜๊ฒฝ ๋ณ€์ˆ˜๊ฐ€ ์—†์„ ๊ฒฝ์šฐ, ๋กœ์ปฌ ๊ฐœ๋ฐœ ํ™˜๊ฒฝ ๋ฐ ์šด์˜/๊ฐœ๋ฐœ ๋„๋ฉ”์ธ์„ ๊ธฐ๋ณธ๊ฐ’์œผ๋กœ ํ—ˆ์šฉํ•˜๋„๋ก ์„ค์ •ํ–ˆ์Šต๋‹ˆ๋‹ค.

์—ฎ์ธ ์ด์Šˆ

  • Closes #

๐Ÿ“ธ ์Šคํฌ๋ฆฐ์ƒท


๐Ÿง‘โ€๐Ÿ’ป ํ…Œ์ŠคํŠธ ๊ฒฐ๊ณผ

  • ๋กœ์ปฌ ํ™˜๊ฒฝ์—์„œ CORS_ORIGINS ํ™˜๊ฒฝ ๋ณ€์ˆ˜ ์„ค์ • ์œ ๋ฌด์— ๋”ฐ๋ฅธ ์˜ค๋ฆฌ์ง„ ํ—ˆ์šฉ ๋™์ž‘ ํ™•์ธ

๐Ÿ“… ์ฒดํฌ๋ฆฌ์ŠคํŠธ

  • ์ดํ•ดํ•˜๊ธฐ ์–ด๋ ค์šด ๋ถ€๋ถ„์— ์ฃผ์„์„ ์ถ”๊ฐ€ํ–ˆ์Šต๋‹ˆ๋‹ค.
  • ๊ด€๋ จ๋œ ๋ฌธ์„œ๋ฅผ ์—…๋ฐ์ดํŠธํ–ˆ์Šต๋‹ˆ๋‹ค.

@soohyunme soohyunme requested a review from yesinkim February 7, 2026 20:02
@soohyunme soohyunme merged commit eecba34 into deploy/cert-develop Feb 11, 2026
2 checks passed
@soohyunme soohyunme deleted the fix/getcloser-security-config branch February 11, 2026 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant